Skip to content

fix: Align Anthropic auto-instrumentation with wrappers#1639

Merged
Luca Forstner (lforst) merged 5 commits intomainfrom
lforst/anthropic-tool-parity
Mar 24, 2026
Merged

fix: Align Anthropic auto-instrumentation with wrappers#1639
Luca Forstner (lforst) merged 5 commits intomainfrom
lforst/anthropic-tool-parity

Conversation

@lforst
Copy link
Member

@lforst Luca Forstner (lforst) commented Mar 23, 2026

This PR brings Anthropic auto-instrumentation in line with the wrapper path and makes the parity contract explicit.

  • Reuse one Anthropic e2e scenario for wrapper and node-hook coverage
  • Snapshot both paths against the same span and payload contract
  • Add streamed tool-use coverage to the shared Anthropic scenario
  • Teach auto-instrumentation to aggregate streamed tool_use blocks fully
  • Favor semantic payload snapshots over merge-row noise so diffs stay obvious

Fixes #1622

@lforst
Copy link
Member Author

Luca Forstner (lforst) commented Mar 24, 2026

Abhijeet Prasad (@AbhiPrasad) I am pretty happy with this now. We can almost use this e2e test scenario as the golden standard for new instrumentation. Most of the added lines come from lockfiles/snapshots.

@lforst Luca Forstner (lforst) merged commit 4e20d33 into main Mar 24, 2026
48 checks passed
@lforst Luca Forstner (lforst) deleted the lforst/anthropic-tool-parity branch March 24, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Anthropic auto-instrumentation plugin drops tool_use output in streaming aggregation

2 participants